[Refactor]: Stagehand MCP becoming the Browserbase MCP #89
Conversation
…and init issues fixed
…tom models in config, session manager migrated to only use stagehand, rm browserbase SDK, remove unnecessary functions
…ics + multi-sessions update in readme
… uses datenow for extra uniqueness
* fix session url bugs * readme small updates + add github workflows to publish + ci
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the MCP server by removing the old Stagehand-only implementation and introducing a unified Browserbase MCP powered by Stagehand.
- Fully deletes the
stagehanddirectory (old MCP implementation). - Adds a new TypeScript-based
src/directory with updated tool definitions, server setup, and CLI. - Updates branding to “Browserbase MCP” across resources, prompts, and documentation.
Reviewed Changes
Copilot reviewed 66 out of 77 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| stagehand/src/tools.ts | Removed old Stagehand tool definitions. |
| stagehand/src/server.ts | Removed legacy Stagehand MCP server logic. |
| src/utils.ts | Updated sanitizeMessage signature and added sanitizeForFilePath. |
| src/tools/index.ts | Newly exports Browserbase Stagehand tools and multi-session tools. |
| src/index.ts | Main MCP server entrypoint using Browserbase and Stagehand. |
| src/program.ts | CLI program to start the new MCP server via Stdio or HTTP transports. |
Files not reviewed (2)
- browserbase/package-lock.json: Language not supported
- stagehand/package-lock.json: Language not supported
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
| "env": { | ||
| "BROWSERBASE_API_KEY": "", | ||
| "BROWSERBASE_PROJECT_ID": "" | ||
| } |
There was a problem hiding this comment.
on local, will they need an LLM api key?
There was a problem hiding this comment.
on local yes you do need (also with any STDIO server thats not the remote hosted url)
package.json
Outdated
| "dependencies": { | ||
| "@browserbasehq/stagehand": "^2.4.0", | ||
| "@modelcontextprotocol/sdk": "^1.13.1", | ||
| "@playwright/test": "^1.49.0", |
There was a problem hiding this comment.
oh yeah we don't need playwright test, can remove.
we're almost fully off of playwright core, just the cookie injection stuff that factory ai wanted
| } | ||
|
|
||
| throw new Error(`Invalid prompt name: ${name}`); | ||
| } |
There was a problem hiding this comment.
we should add some for stagehand, probably good to get some from here:
https://github.com/browserbase/stagehand/blob/main/.cursorrules
| 'key in the action template. For example: {"action": "Fill in the password", "variables": {"password": "123456"}}', | ||
| ), | ||
| }); | ||
|
|
There was a problem hiding this comment.
making a note for the future here, but we should clean up an reuse the zod schemas in https://github.com/browserbase/stagehand/blob/main/lib/inference.ts to avoid maintaining on both sides
There was a problem hiding this comment.
what do you mean? like import schemas from stagehand?
also theres no schema for act in that file
What
Switching the Browserbase MCP to be purely stagehand based.
To be hosted at mcp.browserbase.com/mcp and by smitihery.
Stagehand Primitives as tool calls + new tools to run stagehand sessions in parallel.